Skip to content

Conversation

Henry-Hiles
Copy link
Contributor

@Henry-Hiles Henry-Hiles commented Aug 29, 2025

Purpose

Adds a new Docs instance, as suggested here: #1316

External contributions

Thank you for your contribution! 🎉

Please ensure the following items are checked before submitting your pull request:

  • I have read and followed the contributing guidelines
  • I have read and agreed to the Code of Conduct
  • I have signed off my commits with git commit --signoff (DCO compliance)
  • I have signed my commits with my SSH or GPG key (git commit -S)
  • My commit messages follow the required format: <gitmoji>(type) title description
  • I have added a changelog entry under ## [Unreleased] section (if noticeable change)
  • I have added corresponding tests for new features or bug fixes (if applicable)

@Henry-Hiles
Copy link
Contributor Author

I wasn't sure if I had to signoff and everything for something as simple as this, but I can clone it locally to do that if needed.

@virgile-dev
Copy link
Collaborator

Hey, @Henry-Hiles
Thanks for adding you url and hosting an open instance of Docs !
yes you do need to sign off all changes are subject to it and the CI won’t pass haha

@Henry-Hiles
Copy link
Contributor Author

Hey, @Henry-Hiles Thanks for adding you url and hosting an open instance of Docs ! yes you do need to sign off all changes are subject to it and the CI won’t pass haha

Alrighty, I've signed off.

@virgile-dev
Copy link
Collaborator

I've just re-ran the CI jobs. I think you'll have to add a description to your commit for it to pass.
We'll see @Henry-Hiles :)

@virgile-dev virgile-dev enabled auto-merge (squash) September 2, 2025 04:37
@Henry-Hiles
Copy link
Contributor Author

I've just re-ran the CI jobs. I think you'll have to add a description to your commit for it to pass. We'll see @Henry-Hiles :)

Hi, as I said in the Matrix room, it's an error relating to the CHANGELOG.md, so you need to add the noChangeLog label to this PR :)

@virgile-dev virgile-dev merged commit a799d77 into suitenumerique:main Sep 2, 2025
19 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants